-
Notifications
You must be signed in to change notification settings - Fork 3
Feature: migration signature #149
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: release_2.6-1.4
Are you sure you want to change the base?
Conversation
|
|
||
| CHIP_ERROR DecryptUsingOtaTlvEncryptionKey(MutableByteSpan & block, uint32_t & mIVOffset); | ||
| CHIP_ERROR GetOtaTlvEncryptionKeyId(uint32_t & value) override; | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should the change first go in http://github.com/SiliconLabsSoftware/matter_provisioning/blob/main/src/headers/ProvisionStorage.h ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, but this will be done while implementing the CTM in provisioning. I should have opened this as draft, this is mostly so I can clear the CI in the matter_sdk
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Makes sense,
I can review it again once ready
460cf67 to
951cd59
Compare
|
Is the target branch |
|
@lpbeliveau-silabs Is this still needed? if not can we draft this? |
Problem / Feature
Adding a MigrateAttestationCredentialAPI to allow the Storage migration to be implemented in the matter_sdk.
Change overview
CHIP_ERROR MigrateAttestationCredentialAPI() method added.
Testing
In matter_sdk: Provision -> Build -> Flash -> Read NVM3 to confirm migration.